Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: remove link to ES6 support runners #568

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gthb
Copy link

@gthb gthb commented Oct 22, 2021

Now that tape itself supports ES6, there is no need for “ES6 support with https://www.npmjs.com/package/babel-tape-runner or https://www.npmjs.com/package/buble-tape-runner” anymore, so this item becomes misleading (suggests to the newcomer that tape still doesn't support ES6).

Unless I'm wrong and there is still some reason to use them? If so, maybe specify that reason.

Now that `tape` itself supports ES6, there is no need for “ES6 support with https://www.npmjs.com/package/babel-tape-runner or https://www.npmjs.com/package/buble-tape-runner” anymore, so this item becomes misleading (suggests to the newcomer that `tape` still doesn't support ES6).

Unless I'm wrong and there _is_ still some reason to use them? If so, maybe specify that reason.
@ljharb
Copy link
Collaborator

ljharb commented Oct 22, 2021

tape supports ES Modules via --require, and matches node's ESM support. However, babel-tape-runner and alternatives are still necessary, because every project, forever, should be transpiling to be able to use newer features and not have to pay attention to what features your node version supports.

I agree the caption could be reworded to explain this better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants